Skip to content

docs(web): add description for #theme/Footer and #theme/Metabar#765

Merged
ovflowd merged 1 commit intonodejs:mainfrom
TusharThakur04:docs/custom-components
Apr 16, 2026
Merged

docs(web): add description for #theme/Footer and #theme/Metabar#765
ovflowd merged 1 commit intonodejs:mainfrom
TusharThakur04:docs/custom-components

Conversation

@TusharThakur04
Copy link
Copy Markdown
Contributor

@TusharThakur04 TusharThakur04 commented Apr 12, 2026 •
edited
Loading

Description

The web generator supports 6 overridable #theme/* aliases, but the README only documents 4 of them. This PR adds the missing #theme/Metabar and #theme/Footer entries to the default imports table, introduced in #677.

Validation

No functional changes

Related Issues

Related to #677

Check List

  • I have read the Contributing Guidelines and made commit messages that follow the guideline.
  • I have run node --run test and all tests passed.
  • I have check code formatting with node --run format & node --run lint.
  • I've covered new added functionality with unit tests if necessary.

@TusharThakur04 TusharThakur04 requested a review from a team as a code owner April 12, 2026 16:34
@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 12, 2026 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
api-docs-tooling Ready Ready Preview Apr 12, 2026 4:35pm

Request Review

@cursor
Copy link
Copy Markdown

cursor Bot commented Apr 12, 2026 •
edited
Loading

PR Summary

Low Risk
Low risk documentation-only change that updates the web generator README without affecting runtime behavior.

Overview
Updates the web generator README to fully document the default overridable #theme/* imports by adding the missing #theme/Metabar and #theme/Footer entries (including their defaults and descriptions) to the Default imports table.

Reviewed by Cursor Bugbot for commit 46eb53a. Bugbot is set up for automated code reviews on this repo. Configure here.

@ovflowd ovflowd enabled auto-merge (squash) April 16, 2026 10:30
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 16, 2026 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.43%. Comparing base (3d00fa8) to head (46eb53a).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files 
@@           Coverage Diff           @@
##             main     #765   +/-   ##
=======================================
  Coverage   78.43%   78.43%           
=======================================
  Files         157      157           
  Lines       13962    13962           
  Branches     1152     1152           
=======================================
  Hits        10951    10951           
  Misses       3006     3006           
  Partials        5        5

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ovflowd ovflowd merged commit 524fe0f into nodejs:main Apr 16, 2026
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ovflowd ovflowd ovflowd approved these changes

@avivkeller avivkeller avivkeller approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@TusharThakur04 @ovflowd @avivkeller